Skip to content

Fix bug when empty http proxy env var#18

Merged
christopherjturner merged 1 commit into
mainfrom
fix-empty-http-proxy-bug
Feb 25, 2026
Merged

Fix bug when empty http proxy env var#18
christopherjturner merged 1 commit into
mainfrom
fix-empty-http-proxy-bug

Conversation

@bram2000
Copy link
Copy Markdown
Collaborator

@bram2000 bram2000 commented Feb 25, 2026

Conditionally sets HTTP and HTTPS proxy environment variables if a proxy is configured. If no proxy is specified in the configuration, it explicitly removes these environment variables.

This prevents the application from attempting to use a potentially invalid or non-existent proxy when one is not intended to be used.

@bram2000 bram2000 changed the title AICE-122 refactor tests to make them all function, rather than class Fix bug when empty http proxy env var Feb 25, 2026
Conditionally sets HTTP and HTTPS proxy environment variables if a proxy is configured. If no proxy is specified in the configuration, it explicitly removes these environment variables.

This prevents the application from attempting to use a potentially invalid or non-existent proxy when one is not intended to be used.
@bram2000 bram2000 force-pushed the fix-empty-http-proxy-bug branch from 1c74a15 to 99301d2 Compare February 25, 2026 10:06
@christopherjturner christopherjturner merged commit 75faf96 into main Feb 25, 2026
1 check passed
@christopherjturner christopherjturner deleted the fix-empty-http-proxy-bug branch February 25, 2026 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants